Issue#3828 Refatorar Mesa Diretora by LeandroJatai · Pull Request #3829 · interlegis/sapl

LeandroJatai · 2026-04-14T21:30:03Z

Refatora Mesa Diretora para utilizar o Crud de forma simplifica da criação e edição de mesas e composições

Descrição

Foi criado dois CRUDs para MesaDiretora e ComposicaoMesa, além dos forms correspondentes para criação e edição com regras de negócio ligadas aos dois modelos.
A MesaDiretora foi desligada de SessãoLegislativa e ligada à Legislatura. Os migrations cuidam dessa reassociação.
A tela de acesso público agora seleciona apenas legislatura, TABs são montados com quantas mesas tiver na legislatura selecionada. A navegação dos TABs é transformada em dropdown em telas menores
Após o merge do PR um commit deve ser feito após a execução de npm run build

Issue Relacionada

#3828
Nesta issue estão os requisitos funcionais e não funcionais, além da citação de várias outras issues que envolvem o tema

Motivação e Contexto

Mesas Diretoras nunca tiveram ligação conceitual com sessão legislativa. Sessão Legislativa possui uma definição e existência específica: inicia-se em meados de fevereiro e termina-se em meados de dezembro e diz sobre o intervalo onde haverá sessões plenárias.
Mesa diretora não: Mesa diretora, salvo raríssimas exceções, iniciam-se em 1º de janeiro e conclui-se com um ou dois anos. Dentro das exceções, estão a finalização inesperada de uma mesa e inicio de outra, seja por cassação de algum parlamentar, ou outro motivo qualquer que interrompa uma mesa e inicia-se outra.
Regras de não interseção entre mesas; de contenção em legislatura; de cargo único (que já existia); de só parlamentares da legislatura; de não duplicidade de parlamentar foram colocadas nos forms.

Como Isso Foi Testado?

foi criado o sapl/parlamentares/tests/test_mesadiretora.py que testa os forms

Capturas de Tela (se apropriado):

Tela Pública com 3 Mesas na mesma Legislatura. Nota-se os TABs com as três mesas, onde o Biênio de 23/24 possui duas mesas com datas de inicio e encerramento.

Tela de listagem do CRUDs tradicional para usuário que possui permissão de edição de mesa:

Tela de listagem das composições de uma mesa

Tipos de Mudanças

Bug fix (alteração que corrige uma issue e não altera funcionalidades já existentes)
Nova feature (alteração que adiciona uma funcionalidade e não altera funcionalidades já existentes)
Alteração disruptiva (Breaking change) (Correção ou funcionalidade que causa alteração nas funcionalidades existentes)

Checklist:

Eu li o documento de Contribuição (CONTRIBUTING).
Meu código segue o estilo de código deste projeto.
Minha alteração requer uma alteração na documentação. (Apostila das Oficinas)
Eu atualizei a documentação de acordo.
Eu adicionei testes para cobrir minhas mudanças.
Todos os testes novos e existentes passaram.

…e negócio

edwardoliveira

A motivação conceitual do PR é correta (Mesa Diretora pertence à Legislatura, não à
SessaoLegislativa). Isso é uma das muitas "heranças malditas" da modelagem de dados (mal feita!) do SAPL 2.5 e anteriores. A simplificação do AJAX/jQuery para CRUDs padrão é bem vinda. Os problemas abaixo precisam ser resolvidos antes do merge.

…3fd2a18b5fe8d296b8bc891dc03de0f8595#r3093413994

#3829 (comment) #3829 (comment)

joaohortsenado · 2026-05-04T14:35:01Z

-
-    url(r'^mesa-diretora/remove-parlamentar-composicao/$',
-        remove_parlamentar_composicao, name='remove_parlamentar_composicao'),
+    url(r'^mesa-diretora/', include(


URL renomeada quebra redireciona_urls

A URL sapl.parlamentares:mesa_diretora foi renomeada para mesadiretora_list, mas duas referências fora do diff continuam apontando para o nome antigo:

sapl/redireciona_urls/views.py:34 — parlamentar_mesa_diretora = (app_parlamentares + ':mesa_diretora'). Vai disparar UnknownUrlNameError quando alguém acessar /consultas/mesa_diretora/mesa_diretora_index_html (uso em views.py:495).

sapl/redireciona_urls/tests.py:140 — reverse('sapl.parlamentares:mesa_diretora') quebra a suíte.

Atualizar ambos para mesadiretora_list.

joaohortsenado · 2026-05-04T14:35:01Z

+        migrations.AlterField(
+            model_name='mesadiretora',
+            name='data_inicio',
+            field=models.DateField(verbose_name='Data Início'),


Migration pode falhar em bases legadas

O modelo antigo permitia data_inicio/data_fim nulos em MesaDiretora (null=True). Fluxo atual:

0046 popula essas datas a partir de sessao_legislativa, mas só replica os valores que a sessão tinha.

0048 popula titulo apenas onde data_inicio IS NOT NULL AND data_fim IS NOT NULL.

0049 (esta migration) aplica NOT NULL em ambas.

Se houver MesaDiretora cuja sessão não tinha datas (ou o join falhar), 0049 quebra com IntegrityError no upgrade. Como as bases dos clientes são bem heterogêneas, sugiro:

Em 0046 (ou nova migration intermediária), preencher fallback antes do AlterField para NOT NULL — por exemplo, copiar das datas da legislatura quando a sessão não tiver.

Rodar a migration contra alguns dumps reais antes do release.

joaohortsenado · 2026-05-04T14:35:01Z

+    def __init__(self, *args, **kwargs):
+        super(ComposicaoMesaForm, self).__init__(*args, **kwargs)
+        self.instance.mesa_diretora = self.initial.get('mesa_diretora')
+        self.fields['parlamentar'].queryset = self.fields['parlamentar'].queryset.filter(


ComposicaoMesaForm.__init__ quebra silenciosamente sem initial

Se o form for instanciado sem initial['mesa_diretora'], self.initial.get(...) retorna None e a próxima linha lança AttributeError: 'NoneType' object has no attribute 'legislatura'.

Hoje as views CRUD sempre passam initial, então não dispara em produção, mas qualquer caller futuro (admin, comando, outra view) quebra silenciosamente. Sugestão de guarda:

mesa = self.initial.get('mesa_diretora') if mesa is not None: self.instance.mesa_diretora = mesa self.fields['parlamentar'].queryset = ( self.fields['parlamentar'].queryset .filter(mandato__legislatura=mesa.legislatura) )

joaohortsenado · 2026-05-04T14:35:01Z

-        ultima_filiacao = self.filiacao_set.order_by('-data').first()
+        # este método conta com a ordenação default do model Filiacao para trazer a última filiação primeiro
+        # se order_by for adicionado aqui, o prefetch_related que inclui filiacao_set não irá pré-carregar como esperado
+        ultima_filiacao = self.filiacao_set.first()


filiacao_atual agora depende da ordenação default — vale um teste regressivo

A troca por self.filiacao_set.first() é correta porque Filiacao.Meta.ordering = ('parlamentar', '-data', '-data_desfiliacao'), e o prefetch_related adicionado na ListView depende disso para não disparar query extra. O comentário deixa o motivo claro — ótimo.

Sugestão: adicionar um teste curto que verifique o invariante (criar duas filiações em datas diferentes e checar parlamentar.filiacao_atual). Se um dia alguém mexer no Meta.ordering, o teste pega.

joaohortsenado · 2026-05-04T14:35:02Z

+
+    def clean(self):
+        if self.data_inicio and self.data_fim:
+            if self.data_inicio >= self.data_fim:


Validação >= rejeita mesas de um único dia

Há cenários reais de mesa que dura um único dia (dissolução e recriação no mesmo dia, por cassação ou eventos extraordinários — exatamente o tipo de exceção citado na descrição do PR).

Considerar > em vez de >=, esse requisito precisa ser refinado... o que acha @edwardoliveira

joaohortsenado · 2026-05-04T14:35:02Z

+
+        def get_filterset_kwargs(self, filterset_class):
+            fk = super().get_filterset_kwargs(filterset_class)
+            if 'legislatura' not in self.request.GET and not 'mesa' in self.request.GET:


NIT: not 'mesa' in self.request.GET → 'mesa' not in self.request.GET (PEP 8). Mesma observação na linha de baixo.

joaohortsenado · 2026-05-04T14:35:02Z

-        return _('Mesa da %(sessao)s sessao da %(legislatura)s Legislatura') % {
-            'sessao': self.sessao_legislativa, 'legislatura': self.sessao_legislativa.legislatura
-        }
+        return self.titulo or _('%(legislatura)s - %(data_inicio)s a %(data_fim)s') % {


Datas em __str__ saem em ISO (2021-01-01) em vez do formato pt-br. Como titulo agora é o caminho principal, é raro cair no fallback, mas vale usar django.utils.formats.date_format(d, "SHORT_DATE_FORMAT") para consistência com o resto da aplicação.

joaohortsenado · 2026-05-04T14:35:02Z

+{% endblock actions_search %}
+
+{% block container_table_list %}
+  {% if perms.parlamentares.add_mesadiretora %}


o template usa add_mesadiretora para alternar entre listagem CRUD (admin) e visualização em abas (público). Funcionalmente está consistente com o resto do SAPL. Só ponderando se change_mesadiretora não seria semanticamente mais correto para representar "tem painel de gestão".

joaohortsenado · 2026-05-04T14:35:02Z

+                            {{composicao.parlamentar.nome_parlamentar}}</a>
+                          </div>
+                        </td>
+                        <td>{{composicao.parlamentar.filiacao_atual}}</td>


Partido pode divergir do partido da época

A view antiga (partido_parlamentar_sessao_legislativa) calculava a filiação na data da sessão. Esta agora mostra sempre a filiação mais recente. Para mesas atuais é a mesma coisa; para mesas históricas pode mostrar partido errado (ex.: parlamentar que mudou de partido depois).

É bom validar esse requisito também, @edwardoliveira . Se não for o caso, vale reintroduzir a lógica por data (ex.: método filiacao_em(data) no model).

joaohortsenado · 2026-05-04T14:35:02Z

+        'cargo': new_cargo.id,
+    })
+
+    composicao.refresh_from_db()


a cobertura dos forms está boa. mas acho que tem alguns pontos faltantes:

MesaDiretoraFilterSet: testar que ?mesa=<id> resolve para a legislatura correta e que sem GET cai na legislatura atual; testar 404 para mesa inexistente.

View pública (mesadiretora_filter.html): renderizar a página sem permissão de admin e validar as abas.

test_composicaomesa_form_view_create/update: não fazem assert de status HTTP — adicionar pelo menos assert response.status_code in (200, 302) para detectar erros de form silenciosos.

Variáveis mandato/mandato1/mandato2 criadas mas nunca lidas — substituir por _ = baker.make(...) para deixar a intenção clara.

NIT: faltou newline final no arquivo. (conferir isso nos demais arquivos)

LeandroJatai added 10 commits April 12, 2026 23:57

feat: cria MesaDiretoraCrud e ComposicaoMesaCrud sem forms e regras d…

799f0ec

…e negócio

feat: cria filterset para seleção de legislatura

652e74f

feat: add field titulo

7acb516

feat: impl tabs na listagem de mesa diretora perfil público

2b55d6d

ajustes no layout publico de mesa

b32959c

ajustes de scss para telas mobiles

106847a

altera link para button nas tabs de mesa diretora

08c5eb6

add forms com regras de negócio

2e894ca

add testes para mesa e composição

52240ba

ajustes nos testes e no form mesa diretora

799d0e0

LeandroJatai requested review from edwardoliveira and joaohortsenado April 14, 2026 21:30

remove código substituído

958e83f

edwardoliveira requested changes Apr 16, 2026

View reviewed changes

LeandroJatai added 14 commits April 17, 2026 17:16

fix: #3829 (comment)

cfdfd69

fix: #3829 (comment)

65e50bc

fix: #3829 (comment)

76dec7b

fix: #3829 (comment)

7a14c1e

fix: #3829 (comment)

507cc4a

fix: https://github.com/interlegis/sapl/pull/3829/changes/BASE..958e8…

c9e55dc

…3fd2a18b5fe8d296b8bc891dc03de0f8595#r3093413994

fix: reviews

3d62955

#3829 (comment) #3829 (comment)

fix: ajuste em subnav_mesa.yaml

a6ee8d3

fix: #3829 (comment)

29c9dc0

fix: #3829 (comment)

f36970c

fix: #3829 (comment)

5faefcb

fix: #3829 (comment)

fcb2de6

fix: #3829 (comment)

f777e02

fix: #3829 (comment)

98e687f

LeandroJatai removed the request for review from joaohortsenado April 18, 2026 01:32

fix: ajuste de related_name

1b144e4

LeandroJatai requested a review from edwardoliveira April 18, 2026 01:36

joaohortsenado assigned LeandroJatai May 4, 2026

joaohortsenado requested changes May 4, 2026

View reviewed changes

Conversation

LeandroJatai commented Apr 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Descrição

Issue Relacionada

Motivação e Contexto

Como Isso Foi Testado?

Capturas de Tela (se apropriado):

Tipos de Mudanças

Checklist:

Uh oh!

edwardoliveira left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

LeandroJatai commented Apr 14, 2026 •

edited

Loading

edwardoliveira left a comment •

edited

Loading